Benchmark julia-actions/cache for CI runtime - #24
Merged
Conversation
bernalde
marked this pull request as ready for review
June 8, 2026 09:29
bernalde
commented
Jun 8, 2026
bernalde
left a comment
bernalde
left a comment
Member
Author
There was a problem hiding this comment.
Blocking issues: none.
Nonblocking issues: none.
Questions: none.
Tests run and outcomes:
- Verified fetched PR head
92a0d6939370e914de4d868e1d41be471c17e84bagainst the local checkout. - Verified local merge-base diff matches GitHub-reported totals: 1 changed file, 1 insertion, 0 deletions.
python -c "import pathlib, yaml; yaml.safe_load(pathlib.Path('.github/workflows/ci.yml').read_text())": passed.gh pr checks 24: passed for all three matrix jobs.- Verified CI run
27127936887attempt 1 passed as the cold-cache run and attempt 2 passed as the warm-cache run. - Attempted
julia --project=. -e 'import Pkg; Pkg.test()'locally; it did not reach tests because this checkout's Julia 1.10 environment reports the manifest was resolved with Julia 1.12 and cannot locateOpenSSL_jll. I am not treating that as a PR regression because this PR only changes CI configuration and the GitHub Julia 1.10/1/Windows matrix passed on the PR head.
Issue coverage:
- The change adds only
julia-actions/cache@v3aftersetup-juliaand beforejulia-buildpkg, matching issue 23. - Cold and warm runs confirm cache miss/save, then restore/save, across all matrix jobs.
- Warm critical path improves from the issue baseline
4m17sto2m02s, satisfying the requested material-improvement threshold.
Merge-readiness: merge-ready from my review. This review is submitted as COMMENT because the current GitHub account authored the PR; formal approval must come from another maintainer.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
julia-actions/cache@v3afterjulia-actions/setup-julia@v3and beforejulia-actions/julia-buildpkg@v1.Validation
python -c "import pathlib, yaml; yaml.safe_load(pathlib.Path('.github/workflows/ci.yml').read_text())": passed27127936887attempt 1: passed as the cold-cache run.27127936887attempt 2: passed as the warm-cache run.Benchmark
Baseline from issue 23: run
27072931870.Critical path:
4m17sbaseline to2m02swarm-cache,-135s(-53%).Step Timings
Cache Evidence
No cache foundand thenCache saved successfully.Cache restored successfullyfrom attempt 1 keys, then saved attempt 2 caches.Closes #23